Skip to content

feat(@angular/cli): disable progress when running outside TTY #8501

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 21, 2017

Conversation

devoto13
Copy link
Contributor

The default value is changed from true to progress.stdout.isTTY. It
still has lower priority than value specified by command line flag or in
the .angular-cli.json config.

Fixes #8148

Not sure how to add a test though... Verified manually using:

$ ng build # progress is printed
$ ng build | cat # no progress is printed
$ ng build --progress | cat # progress is printed

The default value is changed from `true` to `progress.stdout.isTTY`. It
still has lower priority than value specified by command line flag or in
the .angular-cli.json config.

Fixes angular#8148
Copy link
Contributor

@filipesilva filipesilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a really nice feature, thanks!

@filipesilva filipesilva self-assigned this Nov 20, 2017
@hansl hansl merged commit 4774049 into angular:master Nov 21, 2017
devoto13 added a commit to devoto13/angular-cli that referenced this pull request Dec 18, 2017
…de TTY

Fixes a shortcoming of angular#8501. Apply same logic for setting --progress
flat when running Karma directly (as opposed to running it using
`ng test` command).

Fixes angular#8148
devoto13 added a commit to devoto13/angular-cli that referenced this pull request Dec 18, 2017
…de TTY

Fixes a shortcoming of angular#8501. Apply same logic for setting --progress
flag when running Karma directly (as opposed to running it using
`ng test` command).

Fixes angular#8148
hansl pushed a commit that referenced this pull request Dec 19, 2017
…de TTY

Fixes a shortcoming of #8501. Apply same logic for setting --progress
flag when running Karma directly (as opposed to running it using
`ng test` command).

Fixes #8148
hansl pushed a commit that referenced this pull request Dec 21, 2017
…de TTY

Fixes a shortcoming of #8501. Apply same logic for setting --progress
flag when running Karma directly (as opposed to running it using
`ng test` command).

Fixes #8148
dond2clouds pushed a commit to d2clouds/speedray-cli that referenced this pull request Apr 23, 2018
…de TTY

Fixes a shortcoming of angular#8501. Apply same logic for setting --progress
flag when running Karma directly (as opposed to running it using
`ng test` command).

Fixes angular#8148
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Don't print progress in a non-TTY context
4 participants